Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(crud): using metadata to update entities based on primary key instead of "found" #622

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dschoeni
Copy link

@dschoeni dschoeni commented Oct 26, 2020

I created this fix for issue #177.

  • Instead of using { ...found } I've added the primary keys to the object, so that only properties from the DTO are actually passed.

Sorry for the messy process, I made some iterations on this to keep the fix as simple as possible.

@dschoeni dschoeni force-pushed the fix/117-eager-breaks-update branch 3 times, most recently from c105d02 to 78c08ab Compare October 26, 2020 17:27
@dschoeni dschoeni marked this pull request as draft October 27, 2020 08:12
@dschoeni dschoeni force-pushed the fix/117-eager-breaks-update branch from 78c08ab to dd9dd51 Compare October 27, 2020 08:26
@dschoeni dschoeni changed the title fix(crud): fixing updateOne to not include existing properties from found upon save fix(crud): using metadata to update entities based on primary key instead of "found" Oct 27, 2020
@dschoeni dschoeni force-pushed the fix/117-eager-breaks-update branch from dd9dd51 to 2fcffae Compare October 27, 2020 08:26
@dschoeni dschoeni marked this pull request as ready for review October 27, 2020 08:51
@dschoeni dschoeni force-pushed the fix/117-eager-breaks-update branch from 2fcffae to 580c19a Compare April 6, 2021 19:29
@rewiko rewiko mentioned this pull request Nov 28, 2021
19 tasks
@rewiko
Copy link
Collaborator

rewiko commented Jan 11, 2022

Released via rewiko#5, https://www.npmjs.com/package/@rewiko/crud version 5.1.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants